[14.0][IMP] spec_driven_model: Verificação no loop de registro dos hooks #2605
Conversation
|
Hi @rvalyi, |
|
Olá @ygcarvalh . Te proponho porém de segurar um pouco esse merge. Vamos ver um pouco até que ponto vc chega primeiro com esses novos módulos de CTe e de MDFe para ter uma visão melhor do big picture do que precisa mudar ou não. Porém claro que se o caminho que vc tá seguindo der certo não vejo problema com esse tipo de mudança. Vc pode fazer um Cherry pick do commit na sua branch nesse tempo. |
|
@felipezago @lfdivino isso ainda é necessário? Para o CT-e e MDF-e? |
|
@ygcarvalh mantem em draft? |
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
|
Hi @rvalyi, |
mileo
force-pushed
the
feat/spec-driven-register-hook
branch
from
1ec5352
to
d15cf52
Compare
github-actions
bot
removed
the
stale
Quando estava implementando o módulo para a CT-e e utilizando o código em #2596, acabei esbarrando em um problema de uma classe que estava sendo utilizada e não possuía nenhum campo com o prefixo implementada. Isso faz com a que a iteração do loop seja interrompida e gere um erro, além de não deixar o módulo ser instalado.
Eu consegui contornar no módulo que está sendo desenvolvido comentando a definição da classe e de onde está o relacional, mas é apenas uma gambiarra para não interromper nada.
A ideia dessa melhoria é que ele só continue com o restante do código, apenas, se qualquer um dos campos contenha esse prefixo.
cc: @mileo @rvalyi @marcelsavegnago @antoniospneto